Skip to content

Comments

Variable length DSS option#1

Open
obonaventure wants to merge 1 commit intomainfrom
variable-length-dss
Open

Variable length DSS option#1
obonaventure wants to merge 1 commit intomainfrom
variable-length-dss

Conversation

@obonaventure
Copy link
Contributor

First attempt to create a variable length DSS option that would support longer data-level length

Copy link
Collaborator

@matttbe matttbe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for this modification, and sorry for the delay.

If I'm remembering well, the original need is to increase the DLL size from 2 to 4 octets, by using the space reserved for the checksum (see Figure 2). So we don't use more option space, the packet size is not modified.

Is the goal here to reduce the overall option space by 4 octets by eventually having the SSN on 2 octets and the DLL on 2 octets, aligned?

If yes, I don't see the point to have many variations: is it not enough to have one bit describing the SSN and DLL sizes?

  • both on 2 octets → we save 4 octets
  • both on 4 octets → same situation as before

WDYT?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants